Skip to content

Reduce cudf-polars test output verbosity - #23480

Merged
rapids-bot[bot] merged 1 commit into
NVIDIA:mainfrom
TomAugspurger:tom/cudf-polars-reasons
Jul 30, 2026
Merged

Reduce cudf-polars test output verbosity#23480
rapids-bot[bot] merged 1 commit into
NVIDIA:mainfrom
TomAugspurger:tom/cudf-polars-reasons

Conversation

@TomAugspurger

Copy link
Copy Markdown
Contributor

Description

This removes the -ra flags from pytest, to just print the count of xfailed/skipped/etc. tests, rather than a reason for each. We added them earlier for debugging, but they're more just getting in the way now.

This removes the `-ra` flags from pytest, to just print the count of
xfailed/skipped/etc. tests, rather than a reason for each. We added them
earlier for debugging, but they're more just getting in the way now.
@TomAugspurger
TomAugspurger requested a review from a team as a code owner July 30, 2026 12:26
@TomAugspurger
TomAugspurger requested a review from msarahan July 30, 2026 12:26
@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 3baec47e-c0e4-434a-bc7f-dcb52887acb6

📥 Commits

Reviewing files that changed from the base of the PR and between 6933422 and f0b03a8.

📒 Files selected for processing (2)
  • ci/run_cudf_polars_polars_tests.sh
  • ci/test_python_other.sh
💤 Files with no reviewable changes (1)
  • ci/run_cudf_polars_polars_tests.sh

📝 Walkthrough

Summary by CodeRabbit

  • Tests
    • Updated automated test commands to produce clearer runtime information by focusing on the slowest tests.
    • Preserved existing test coverage, selection logic, and execution behavior.

Walkthrough

Changes

cudf-polars CI test commands

Layer / File(s) Summary
Remove pytest -ra arguments
ci/run_cudf_polars_polars_tests.sh, ci/test_python_other.sh
Three cudf-polars pytest invocations no longer pass -ra; existing test selection, execution, and duration options remain unchanged.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

Suggested reviewers: msarahan

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: reducing cudf-polars pytest output verbosity.
Description check ✅ Passed The description matches the change by explaining that pytest -ra flags were removed from cudf-polars tests.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Fix failing CI checks
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@TomAugspurger TomAugspurger added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 30, 2026
@TomAugspurger

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot
rapids-bot Bot merged commit 4886ffe into NVIDIA:main Jul 30, 2026
258 of 263 checks passed
@TomAugspurger
TomAugspurger removed the request for review from msarahan July 30, 2026 17:56
@TomAugspurger
TomAugspurger deleted the tom/cudf-polars-reasons branch July 30, 2026 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants